-
-
Notifications
You must be signed in to change notification settings - Fork 355
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Spark #1360
Merged
Merged
Spark #1360
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* QT: Amount column size fix * QT: Status column size fix
* integrate spark * update default of Requested payments history, combine mintspark trans, update nTxFee * update spark address book * fix bug for spark address book * add default spark address into spark address book * update migrateLelantusToSpark * fix spendspark crash * update typo * integrate Anonymize All * merge remote branch firo * fix migrateLelantusToSpark * remove close X of the migrate dialog * remove close X of the private dialog * fix the migration issues * fix the migration issues * update for verifying migration * update for verifying migration * update calculating estimated fee of spendspark * fix mintspark/spendspark from an another wallet * fix crash if using the existing wallet * fix bug 1282 & 1288 * Add output spark address & amount * Merge remote-tracking branch 'firo/spark' into spark * remove i * remove addresstype * fix edit spark label, size of address col in transaction history * Version bump * Fix Spark address view on recieverequestdialog * Merge conflict fixed * Review commit resolved * Remove sensitive data from Coin --------- Co-authored-by: levonpetrosyan93 <petrosyan.levon93@gmail.com> Co-authored-by: levonpetrosyan93 <45027856+levonpetrosyan93@users.noreply.github.com>
* Disable building libconsensus for Windows in CI * setuptools workaround for macOS
* Spark address/key serialization functions implemented * CSparkWallet and mint db functions added * Spark mint tx creation, GetAvailableSparkCoins() implemented * Bind serial commitments to chain context * Use default curve generator for spend component * Completed merge * Use default address when auto minting * Adding domain separators when doing hashes * Adding version and checksum in address serialization * Saving serial_context into CSparkMintMeta * Moving Spark related code into separate directory * Embed KDF derived key size into hash instantiation * Generalize range proofs to arbitrary set sizes * Spark state implemented * More state functionality implemented * WIP: spend verification batching * Adding Spark state into mempool * Use a key-committing AEAD construction * More spark state functionality implemented * Spark balance proof fixed * Adds an encoding method for address scrambling * Switch from hex to `bech32m` encoding * Add tests * SpendTransaction refactored and added serialization for it * Review comment applied and fixed a bug in mint generation * Spark spend creation * Spark spend verification * More state functionality implemented and bug fixes * Some bug fixes and cleanup * Bug fixes and review comments resolved * More review comments resolved * Adding several rpc calls for spark * Adding more rpc calls * Failing unittests fixed * Version bump * Devnet Spark HF block * Added fuzz folder for persistent fuzzing libspark fuzzing * bech32 fuzzing harness for honggfuzz added * fix bech32_fuzz.cpp * Added f4grumble fuzzing harness * Forgot to add return statement to f4grumble_fuzz.cpp * Moved libspark related fuzzing to its own folder * Added FuzzedDataProvider.h * Integrated FuzzedDataProvider.h in bech32_fuzz.cpp * Fixed fuzzing enum in bech32_fuzz.cpp * missing semicolon * Added fuzz folder for persistent fuzzing libspark fuzzing bech32 fuzzing harness for honggfuzz added Added f4grumble fuzzing harness Revert "missing semicolon" This reverts commit 330cff6. Fuzzing directory cleanup Added FuzzedDataProvider.h * Added fuzzing utilities for group elements and scalars * Added fuzzing for aead.cpp * Added vector versions of GetGroupElement and GetScalar in fuzzing_utilities.cpp * Added single proof fuzz test for bpplus_fuzz * LelantusToSpark function implemented * Refactoring of file paths * lelantustospark rpc name refactored * Adding check to stop lelantus on consensus level * Added fuzz folder for persistent fuzzing libspark fuzzing bech32 fuzzing harness for honggfuzz added Added f4grumble fuzzing harness Revert "missing semicolon" This reverts commit 330cff6. Fuzzing directory cleanup Added FuzzedDataProvider.h * Added fuzzing utilities for group elements and scalars * Added fuzzing for aead.cpp * Added vector versions of GetGroupElement and GetScalar in fuzzing_utilities.cpp * Added single proof fuzz test for bpplus_fuzz * Changed relative paths of dependencies in secp256k1/src/cpp * More relative path changes in src/secp256k1 * Changed relative paths in crypto/common.h * Removed FuzzedSecp256k1Object deconstructor * Addedd aead_fuzz.cpp and other utilities Adding check to stop lelantus on consensus level Added fuzz folder for persistent fuzzing libspark fuzzing bech32 fuzzing harness for honggfuzz added Added f4grumble fuzzing harness Revert "missing semicolon" This reverts commit 330cff6. Fuzzing directory cleanup Added FuzzedDataProvider.h Added fuzzing utilities for group elements and scalars Added fuzzing for aead.cpp * Added vector versions of GetGroupElement and GetScalar in fuzzing_utilities.cpp * Added single proof fuzz test for bpplus_fuzz * Changed relative paths of dependencies in secp256k1/src/cpp * More relative path changes in src/secp256k1 * Changed relative paths in crypto/common.h * Removed FuzzedSecp256k1Object deconstructor * Added batch bpplus proofs fuzzing tests * Changed relative paths in libspark/params.h * Fixed errors in bpplus_fuzz.cpp * Fixed more errors * More fixes in bpplus.cpp * Completed bpplus_fuzz tests Added batch bpplus proofs fuzzing tests Changed relative paths in libspark/params.h Fixed errors in bpplus_fuzz.cpp Fixed more errors More fixes in bpplus.cpp Changed type of sizes in bpplus_fuzz.cpp * Added chaum_fuzz.cpp * Changed relative paths in src/wallet/crypter.h * Changed relative path in arith_uint256.h exit * Completed chaum fuzz tests Added chaum_fuzz.cpp Changed relative paths in src/wallet/crypter.h Changed relative path in arith_uint256.h exit * Added schnorr_fuzz.cpp * Addedd grootle_fuzz.cpp * Added length to ConsumeBytes call * Initialized sizes vector via a loop in grootle_fuzz.cpp * Changed relative paths in support/allocator/secure.h * Changed relative paths in support/allocator/secure.h again * Added mint_transaction_fuzz.cpp * Changed type of parameter in MintTransaction in mint_transaction_fuzz.cpp * Changed relative path in params.cpp * Changed relative paths in consensus/params.h * Changed relative paths in primitives/block.h * Changed relative paths in primitives/transaction.h * Changed relative paths in src/script/script.h * Changed more relative paths in src/script/script.h * Changed relative paths in crypto/MerkleTreeProof/mtp.h * Changed relative paths in crypto/MerkleTreeProof/mtp.h. Actually this time. * Changed relative paths in crypto/progpow.h * Completed mint_transaction_fuzz.cpp Added mint_transaction_fuzz.cpp Changed type of parameter in MintTransaction in mint_transaction_fuzz.cpp Changed relative path in params.cpp Changed relative paths in consensus/params.h Changed relative paths in primitives/block.h Changed relative paths in primitives/transaction.h Changed relative paths in src/script/script.h Changed more relative paths in src/script/script.h Changed relative paths in crypto/MerkleTreeProof/mtp.h Changed relative paths in crypto/MerkleTreeProof/mtp.h. Actually this time. Changed relative paths in crypto/progpow.h * Added spend_transaction_fuzz.cpp * Fixed errors in spend_transaction_fuzz.cpp * Completed spend_transaction_fuzz.cpp Added spend_transaction_fuzz.cpp Fixed errors in spend_transaction_fuzz.cpp * Updated relative paths in MultiExponent.cpp * fuzzing and coverage of different libspark files * makefile and code-coverage for libspark * spend transaction modified * coverage, input, results removed * empty and temporary files deleted * empty files removed * readme for fuzzing modified * original paths restored and conflict resolved, flags added in fuzz makefile * path fixed and merged latest spark * path fixed * instructions for installing dependencies added in fuzz readme * binary removed and loop length resized * readme modified & vetor size reserved in fuzzing_utilities --------- Co-authored-by: levonpetrosyan93 <petrosyan.levon93@gmail.com> Co-authored-by: Aaron Feickert <66188213+AaronFeickert@users.noreply.github.com> Co-authored-by: Mikerah <mikerah@hashcloak.com> Co-authored-by: Onur İnanç <e191322@metu.edu.tr> Co-authored-by: HashCloak <info@hashcloak.com>
* Avoid catch(...) when possible * Add wrappers around major sigma/lelantus/spark calls to catch exceptions * Catch exception thrown from CheckLelantusJMintTransaction * Bug fix * Fixed tests * Disable failing elysium test
justanwar
approved these changes
Nov 21, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What brings this PR